Skip to content

Upload code for review#645

Closed
m1nh4ke wants to merge 3 commits intoTheCSharpAcademy:mainfrom
m1nh4ke:main
Closed

Upload code for review#645
m1nh4ke wants to merge 3 commits intoTheCSharpAcademy:mainfrom
m1nh4ke:main

Conversation

@m1nh4ke
Copy link

@m1nh4ke m1nh4ke commented Jan 31, 2026

No description provided.

m1nh4ke and others added 3 commits January 31, 2026 21:13
Refactor input handling to prevent null values and improve error checking.
@TheCSharpAcademy
Copy link
Owner

TheCSharpAcademy commented Feb 7, 2026

@m1nh4ke Project approved! 😄✅ And well done for completing the challenges! I recommend you always do that, as it pushes you and makes you learn faster.

🥋And congratulations! You got your green belt!! 🟩🟩

Feedback
🔍️Your code is super clean and well organized, I'm really glad you used only one method for all games. One thing I should notice is that the Random() class can be a bit less random if instantiated every game. Ideally you'd reuse one Random instance for everything

🔍Small, not super important detail, I think it would have been better to generate a history per game, instead of per question.

🔍Your console get's a bit cluttered as operations pile up. It's good in console apps to clear the UI after each operation.

🔍It's ok to use LLMs to help you, but remove the clutter they leave behind before submitting your projects. Most of their comments are unnecessary.  You can read our code conventions here🤓.

image

👪Oh and if possible join our Discord Community and add your username to your profile so you can be part of the community as well. 😁 And do you mind adding a country in your profile as well? The dashboard and leaderboard look neater when there's a flag. If you're not comfortable with that you can submit a random country.

☕🍵 If you think we're being helpful, consider contributing with a donation, any amount help us keep our free service constantly improving.

✋🏻Looking forward to seeing your next projects!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants